Skip to content

Added support for Model Context Protocol #8480

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 18 commits into from
Aug 15, 2025
Merged

Conversation

glen-84
Copy link
Collaborator

@glen-84 glen-84 commented Jul 31, 2025

Summary of the changes (Less than 80 chars)

  • Added support for Model Context Protocol.

Related to #8197


📃 To-do:

  • Fix the required property for output schemas (⛔ waiting for new "IsDirectConditional" or similar API).
  • Endpoint routing "MapGraphQLMcp".
  • OperationCompiler object pool (?). (⛔ waiting to discuss)
  • Add additionalProperties: true/false to input schema. (⛔ waiting to discuss access to InputParserOptions)
  • Output schema.
  • Send tools/list_changed notification.
  • Add tests for string patterns.
  • Auth
  • $list: [String!]! # = ["default"] FIXME: The validation might be broken.
  • FIXME: It should not be necessary to provide this variable.
  • Documentation
  • TODO: Treating all unknown scalar types as strings is a temporary solution.
  • Add support for decimal and long types serialized as strings.

❓ Questions

  • Is GetGraphQLType correct? Is there another way to do this?

📝 Notes

  • I've made the format of the TimeSpanType public so that this information can be accessed externally.
  • File input is not supported. This requires additional planning.

@Aniruddh25
Copy link

Aniruddh25 commented Aug 14, 2025

curious: @glen-84, @michaelstaib when is this planned to be merged and released?

@glen-84 glen-84 marked this pull request as ready for review August 15, 2025 09:25
@glen-84 glen-84 merged commit eee03da into main Aug 15, 2025
108 checks passed
@glen-84 glen-84 deleted the gai/model-context-protocol branch August 15, 2025 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants